Skip to content

chore(Konflux): Switch Konflux application#140

Merged
mtodor merged 1 commit into
mainfrom
mtodor/change-konflux-app-name
Jun 25, 2026
Merged

chore(Konflux): Switch Konflux application#140
mtodor merged 1 commit into
mainfrom
mtodor/change-konflux-app-name

Conversation

@mtodor

@mtodor mtodor commented Jun 25, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator

Description

We have changed Konflux application name to be aligned with product name.
This PR: https://gitlab.cee.redhat.com/releng/konflux-release-data/-/merge_requests/19419

This PR is switching to correct Konflux application.

Validation

  • Trigger Konflux builds in PR

@mtodor mtodor requested a review from janisz as a code owner June 25, 2026 09:57
@coderabbitai

coderabbitai Bot commented Jun 25, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

Updates the appstudio.openshift.io/application annotation in two Tekton PipelineRun manifests to agentic-cluster-security-suite.

Changes

Tekton PipelineRun annotation update

Layer / File(s) Summary
Annotation value update
.tekton/acs-mcp-server-pull-request.yaml, .tekton/acs-mcp-server-push.yaml		 Both PipelineRun manifests set appstudio.openshift.io/application to agentic-cluster-security-suite.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main change: switching the Konflux application name.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description check ✅ Passed The description matches the PR changes by explaining the Konflux application name update to the product-aligned value.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch mtodor/change-konflux-app-name

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@codecov-commenter

codecov-commenter commented Jun 25, 2026 •
edited
Loading

Copy link
Copy Markdown

❌ 2 Tests Failed:

Tests completed Failed Passed Skipped
380 2 378 12
View the full list of 2 ❄️ flaky test(s) 
::policy 1

Flake rate in main: 100.00% (Passed 0 times, Failed 48 times)

Stack Traces | 0s run time 
- test violation 1
- test violation 2
- test violation 3
::policy 4

Flake rate in main: 100.00% (Passed 0 times, Failed 48 times)

Stack Traces | 0s run time 
- testing multiple alert violation messages 1
- testing multiple alert violation messages 2
- testing multiple alert violation messages 3

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

@github-actions

github-actions Bot commented Jun 25, 2026

Copy link
Copy Markdown

E2E Test Results

Commit: d8ebce6
Workflow Run: View Details
Artifacts: Download test results & logs

=== Evaluation Summary ===

  ✓ cve-detected-clusters (assertions: 3/3)
  ✓ cve-clusters-general (assertions: 3/3)
  ✓ cve-cluster-does-not-exist (assertions: 3/3)
  ✓ cve-detected-workloads (assertions: 3/3)
  ✓ rhsa-not-supported (assertions: 2/2)
  ✓ cve-cluster-list (assertions: 3/3)
  ✓ cve-cluster-does-exist (assertions: 3/3)
  ✓ cve-log4shell (assertions: 3/3)
  ✓ list-clusters (assertions: 3/3)
  ✓ cve-multiple (assertions: 3/3)
  ✗ cve-nonexistent (assertions: 3/3)
      one or more verification steps failed

Tasks:      10/11 passed (90.91%)
Assertions: 32/32 passed (100.00%)
Tokens:     ~49352 (estimate - excludes system prompt & cache)
MCP schemas: ~12562 (included in token total)
Agent used tokens:
  Input:  13692 tokens
  Output: 19276 tokens
Judge used tokens:
  Input:  41442 tokens
  Output: 35610 tokens

@mtodor mtodor merged commit 01e8642 into main Jun 25, 2026
11 checks passed
@mtodor mtodor deleted the mtodor/change-konflux-app-name branch June 25, 2026 11:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@janisz janisz janisz approved these changes

Assignees

No one assigned

Labels

konflux-build

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@mtodor @codecov-commenter @janisz